sequenceDiagram
  participant CLI
  participant Init/Update
  participant CodexAdapter
  participant FS
  participant Env as Environment

  CLI->>Init/Update: generate/update commands (cmd.path)
  Init/Update->>CodexAdapter: ToolCommandAdapter.getFilePath(commandId)
  CodexAdapter->>Env: read CODEX_HOME
  Env-->>CodexAdapter: CODEX_HOME or undefined
  CodexAdapter->>CodexAdapter: getCodexHome() -> resolve path (CODEX_HOME or os.homedir()+/.codex)
  CodexAdapter-->>Init/Update: return absolute file path for Codex command
  Init/Update->>Init/Update: if path is absolute use as-is else join with project path
  Init/Update->>FS: write command file to resolved path
  FS-->>Init/Update: write result
  Init/Update-->>CLI: completion

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Review Complete

Your review story is ready!

View Story

Comment !reviewfast on this PR to re-generate the story.

Greptile Overview

Greptile Summary

This PR adds support for global path installation in the Codex adapter, allowing commands to be installed to ~/.codex/prompts/ (or $CODEX_HOME/prompts/) instead of the project directory. This aligns with Codex's design where custom prompts are global rather than project-scoped.

Key changes:

• Codex adapter now returns absolute paths via os.homedir() with CODEX_HOME env var support
• init.ts and update.ts handle absolute paths with path.isAbsolute() guards before joining with project path
• Type definitions and specs updated to reflect that adapters can return absolute paths
• Comprehensive test coverage for absolute path behavior and environment variable override
• Documentation updated to clearly indicate Codex's global installation pattern with footnote

The implementation is clean, well-tested, and backwards-compatible. The path.isAbsolute() check ensures other adapters that return relative paths continue to work correctly.

Confidence Score: 5/5

• This PR is safe to merge with no identified issues
• Implementation is well-designed with proper absolute path handling, comprehensive test coverage including env var override scenarios, and updated documentation. The path.isAbsolute() guard ensures backwards compatibility with existing adapters.
• No files require special attention

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant User
    participant InitCommand
    participant UpdateCommand
    participant CommandAdapterRegistry
    participant CodexAdapter
    participant FileSystemUtils
    participant OS

    User->>InitCommand: opsx init --tools codex
    InitCommand->>CommandAdapterRegistry: get('codex')
    CommandAdapterRegistry-->>InitCommand: codexAdapter
    InitCommand->>CodexAdapter: getFilePath('explore')
    CodexAdapter->>OS: process.env.CODEX_HOME?
    alt CODEX_HOME set
        OS-->>CodexAdapter: /custom/codex-home
        CodexAdapter->>CodexAdapter: path.resolve(CODEX_HOME)
    else CODEX_HOME not set
        CodexAdapter->>OS: os.homedir()
        OS-->>CodexAdapter: /home/user
        CodexAdapter->>CodexAdapter: path.join(homedir, '.codex')
    end
    CodexAdapter-->>InitCommand: /home/user/.codex/prompts/opsx-explore.md
    InitCommand->>InitCommand: path.isAbsolute(cmd.path)?
    alt path is absolute
        InitCommand->>FileSystemUtils: writeFile(absolutePath, content)
    else path is relative
        InitCommand->>FileSystemUtils: writeFile(projectPath + relativePath, content)
    end
    FileSystemUtils->>FileSystemUtils: createDirectory(dirname)
    FileSystemUtils->>OS: fs.writeFile()
    FileSystemUtils-->>InitCommand: Success
    InitCommand-->>User: Codex commands installed to ~/.codex/prompts/